Skip to content

Claude/fix effects count display 2z de5#708

Merged
braedonsaunders merged 2 commits intoclaude/build-openstudio-app-kgia5from
claude/fix-effects-count-display-2zDe5
Jan 6, 2026
Merged

Claude/fix effects count display 2z de5#708
braedonsaunders merged 2 commits intoclaude/build-openstudio-app-kgia5from
claude/fix-effects-count-display-2zDe5

Conversation

@braedonsaunders
Copy link
Owner

No description provided.

claude added 2 commits January 6, 2026 00:31
…gs with DAW UI

- Fix effects count badge in track header to count all 35 effects
  instead of just 5 standard ones
- Update native bridge types to match all frontend effect settings:
  - Wah: Add rate, depth, baseFrequency, maxFrequency, sensitivity, mix
  - FrequencyShifter: Add direction enum
  - GranularDelay: Add pitchRandom, position, positionRandom, reverse, freeze
  - RotarySpeaker: Add distance, brake speed mode
  - AutoPan: Add width
  - MultiFilter: Add drive, lfoWaveform, envelopeSensitivity, keyTrack, etc
  - Vibrato: Add stereo, tempoSync, subdivision
  - StereoImager: Add midLevel, sideLevel, bassMonoAmount, balance
  - Exciter: Add harmonics enum (odd/even/both)
  - MultibandCompressor: Add low/mid/high band settings with proper field names
  - StereoDelay: Add crossFeed, tone, pingPong
  - Harmonizer: Add voices, spread, shift, keyLock, customIntervals
The reverb feedback formula was treating decay (0.1-10 seconds) as if it
were a 0-1 range value. With default decay of 1.5 seconds, feedback was
calculated as 1.33 (>1), causing exponential gain buildup.

Fixed by:
- Normalizing decay to 0-1 range before feedback calculation
- Capping feedback at 0.93 for stability
@vercel
Copy link

vercel bot commented Jan 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
openstudio Ready Ready Preview, Comment Jan 6, 2026 1:18am

@braedonsaunders braedonsaunders merged commit 71f7c5f into claude/build-openstudio-app-kgia5 Jan 6, 2026
2 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants